Conversation
adespawn
force-pushed
the
arm-release
branch
2 times, most recently
from
ca9b1ac to
b281eea
Compare
There was a problem hiding this comment.
Pull request overview
This PR adds ARM (aarch64-unknown-linux-gnu) architecture support to the release process. The changes enable building, testing, and publishing ARM64 binaries alongside the existing x86_64 binaries.
Changes:
- Added aarch64-unknown-linux-gnu as a build target in package.json
- Created a new build job for aarch64 architecture using ubuntu-24.04-arm runners
- Added an aarch64 test job that runs examples (integration tests are not run on ARM due to platform limitations)
- Updated the publish job to include ARM artifacts and wait for ARM tests to complete
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| package.json | Added aarch64-unknown-linux-gnu to the napi targets array for multi-architecture support |
| .github/workflows/release.yml | Added build and test jobs for ARM64 architecture, removed outdated comments, and updated publish job dependencies and artifact handling |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@wprzytula Some thought here: since we are releasing multiple packages (1 main + 1 / architecture) maybe we want to have a separate step that releases all the packages in the Doing dry run should be able to catch most of the problems from our end, so (in theory) the only chance the full run could fail later, would be infrastructure problems: ex. npm dying in the middle of release process |
adespawn
force-pushed
the
arm-release
branch
4 times, most recently
from
cee36df to
a19b65e
Compare
|
I've tested all the changes in the dry mode: https://github.com/scylladb/nodejs-rs-driver/actions/runs/22901912473 |
Yes, sounds wise. |
Add arm to release process. This includes building, and then testing the built binary on examples (with a single node version).
I've testes this CI with
--dry-runoption, to ensure everything works correctly.